feat: implement Browse AI plugin against official v2 API - #1042
Conversation
|
Someone is attempting to deploy a commit to the corsair Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (8)
Included review availability: Your plan provides up to 10 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughAdds a new Browse AI Corsair provider with versioned schemas, authenticated v2 API requests, endpoint operations, audit logging, error handling, package configuration, and automated tests. ChangesBrowse AI provider
Estimated code review effort: 4 (Complex) | ~45 minutes Merge Risk: ⚪ Minimal · up to This PR adds the Browse AI plugin and has no actionable merge-blocking risk remaining based on the supplied evidence; it is merge-ready after normal checks and review. Sequence Diagram(s)sequenceDiagram
participant CorsairPlugin
participant EndpointOperation
participant browseaiCall
participant makeBrowseaiRequest
participant BrowseAIAPI
CorsairPlugin->>EndpointOperation: Invoke typed operation
EndpointOperation->>browseaiCall: Validate input and pass schema
browseaiCall->>makeBrowseaiRequest: Pass API key and request options
makeBrowseaiRequest->>BrowseAIAPI: Send v2 HTTP request
BrowseAIAPI-->>makeBrowseaiRequest: Return JSON response or HTTP error
makeBrowseaiRequest-->>browseaiCall: Return parsed result or error
browseaiCall-->>EndpointOperation: Return typed result
EndpointOperation-->>CorsairPlugin: Log completed audit event
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Linked Issues checkExplanation The PR implements the requested BrowseAI integration with data-analysis operations across system, robot, task, monitor, and webhook resources. It also includes webhook support through create and list operations, satisfying issue Full details: Out of Scope Changes checkExplanation The changes are consistent with the BrowseAI integration objective in issue
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThe PR adds a Browse AI provider plugin backed by the official v2 API.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the previously reported output-validation bypass and non-idempotent write replay have both been addressed. Important Files Changed
Sequence DiagramsequenceDiagram
participant App as Corsair application
participant Plugin as Browse AI plugin
participant Client as Browse AI client
participant API as Browse AI v2 API
App->>Plugin: Invoke typed endpoint
Plugin->>Plugin: Parse input schema
Plugin->>Client: Request with endpoint output schema
Client->>API: Bearer-authenticated HTTP request
API-->>Client: JSON response
Client->>Client: Parse output schema
Client-->>Plugin: Validated result
Plugin-->>App: Typed response
Reviews (2): Last reviewed commit: "chore: merge main into feat/browserai-pl..." | Re-trigger Greptile |
Plugin PR scorecard —
|
| Check | Status | Notes |
|---|---|---|
| R1 — Scope: plugin files only | ✅ | |
| R2 — Tests with assertions | ✅ | |
| R3 — Description complete | ✅ | |
| R3 — Linked issue / claim | ✅ | |
| R4 — Demo video / recording | ✅ |
Rules: PLUGIN_PR_RULES.md · re-runs on every push
|
Hey @TanayGurav19, thanks for the contribution! 🏴☠️ Before a maintainer reviews, please fix the items below — the review re-runs automatically on your next push. Must fix
Rule Used: Every endpoint must validate inputs and outputs wi... (source) Knowledge Base Used: Provider plugin implementation conventions
Knowledge Base Used: Provider plugin implementation conventions If anything remains after your next push, a maintainer will take it from there and do the final review and merge. |
Maintainer review neededAutomated rounds are exhausted. Remaining findings:
Rule Used: Every endpoint must validate inputs and outputs wi... (source) Knowledge Base Used: Provider plugin implementation conventions
Knowledge Base Used: Provider plugin implementation conventions |
Dhirenderchoudhary
left a comment
There was a problem hiding this comment.
LGTM tested locally
Description
Adding Browseai Plugin For corsair.
Closes #1040
Checklist
Before submitting your PR, please verify the following:
pnpm lintand all checks passpnpm typecheckand there are no TypeScript errorspnpm buildand all packages build successfullypnpm testand all tests passScreenshots / Demos (if applicable)
Additional Notes
Summary by CodeRabbit